Skip to content

Conversation

@cybertale
Copy link
Contributor

The last commit #14242 changes FXOS8700 to use new DT defines in its
dts_fixup.h, and the build was also succeeded after it. But
DT_NXP_FXOS8700_0_INT2_GPIOS_CONTROLLER and
DT_NXP_FXOS8700_0_INT2_GPIOS_PIN were missed so the build_all project
is still failing.

Signed-off-by: Song Qiang [email protected]

The last commit changes FXOS8700 to use new DT defines in its
dts_fixup.h, and the build was also succeeded after it. But
DT_NXP_FXOS8700_0_INT2_GPIOS_CONTROLLER and
DT_NXP_FXOS8700_0_INT2_GPIOS_PIN were missed so the build_all project
is still failing.

Signed-off-by: Song Qiang <[email protected]>
@cybertale
Copy link
Contributor Author

Hi @galak , the last time I fix this problem in #14242 I didn't use sensors_a_h.conf and sensors_trigger_a_h.conf as configuration file but enabled the driver of fxos8700 manually. So I missed two defines, which are being added in this commit.

@codecov-io
Copy link

Codecov Report

Merging #14781 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master   #14781   +/-   ##
=======================================
  Coverage   52.51%   52.51%           
=======================================
  Files         309      309           
  Lines       45048    45048           
  Branches    10419    10419           
=======================================
  Hits        23656    23656           
  Misses      16584    16584           
  Partials     4808     4808

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3ebd9b4...fb6c0f5. Read the comment docs.

@galak galak added the bug The issue is a bug, or the PR is fixing a bug label Mar 28, 2019
@galak galak added this to the v1.14.0 milestone Mar 28, 2019
@galak galak merged commit cb26591 into zephyrproject-rtos:master Mar 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug The issue is a bug, or the PR is fixing a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants